[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: mudit06mah
Once this PR has been reviewed and has the lgtm label, please assign joaquimrocha for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

@skoeva Please confirm if PR: #4382 has to be merged as well :)

Thanks for this. But… sorry, can you please have one PR per issue?

I would suggest to try to get one PR merged first so you can learn how things are done in this project before continuing on with other ones.

Thanks for this. But… sorry, can you please have one PR per issue?

@skoeva asked me to create an atomic PR instead of individual ones:
#4402 (review)

My bad for not adding proper commit messages, should I redo these PRs with appropriate commit messages?

We do atomic commits - you can do an interactive rebase (git rebase -i) to squash your commits under each PR into one. The contribution guidelines should be helpful

@illume @skoeva it appears that I have misunderstood the word atomic, I thought it meant to create a big single PR (T_T)
I would like to apologize for this misunderstanding and redo my commits.